Skip to content

Conversation

@dcandler
Copy link
Collaborator

The cmake scripts to fetch content such as llvm-project and picolibc can be called by both the top-level project as well as the sub-projects. To prevent the content being checked out and patched repeatedly, the FETCHCONTENT_SOURCE_DIR_<PROJECT> variables should be defined as these will override the declared source on any subsequent calls.

These variables are already passed down from top-level to sub-project, however the variables themselves are never defined unless configured by a user. This patch addresses this by setting the variables inside the fetch scripts as soon as the fetch content is declared.

The cmake scripts to fetch content such as llvm-project and picolibc
can be called by both the top-level project as well as the
sub-projects. To prevent the content being checked out and patched
repeatedly, the FETCHCONTENT_SOURCE_DIR_<PROJECT> variables should
be defined as these will override the declared source on any
subsequent calls.

These variables are already passed down from top-level to
sub-project, however the variables themselves are never defined
unless configured by a user. This patch addresses this by setting
the variables inside the fetch scripts as soon as the fetch
content is declared.
@dcandler dcandler requested a review from statham-arm November 27, 2024 13:32
@dcandler dcandler merged commit 4dd4c4d into ARM-software:main Nov 27, 2024
1 check passed
@dcandler dcandler deleted the fix_fetchcontent branch November 27, 2024 13:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants